Merged
Conversation
After the DataManager cache check, run a DESCRIBE query against the app's in-memory OntModel (full imports closure) before falling through to the external proxy or 405. Covers both slash-based term URIs (e.g. schema:category) and hash-based namespaces (e.g. sioc:ns → describes all sioc:ns#* terms via the WHERE clause). This allows facet headers and other term lookups to resolve from imported ontologies without requiring ENABLE_LINKED_DATA_PROXY=true. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Creates two OWL classes in a made-up hash-based namespace, clears the in-memory ontology, then retrieves the namespace document via ?uri= and verifies both class descriptions are returned. The namespace is not DataManager-mapped and not a registered app, so the request exercises the new OntModel DESCRIBE path in ProxyRequestFilter. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Set filter.ontology = Optional.empty() in setUp() so existing tests reach the proxy/NotAllowed paths they were designed to test. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
namedgraph
added a commit
that referenced
this pull request
Apr 11, 2026
* Serve namespace ontology terms from ProxyRequestFilter After the DataManager cache check, run a DESCRIBE query against the app's in-memory OntModel (full imports closure) before falling through to the external proxy or 405. Covers both slash-based term URIs (e.g. schema:category) and hash-based namespaces (e.g. sioc:ns → describes all sioc:ns#* terms via the WHERE clause). This allows facet headers and other term lookups to resolve from imported ontologies without requiring ENABLE_LINKED_DATA_PROXY=true. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Add proxy test for OntModel namespace term lookup Creates two OWL classes in a made-up hash-based namespace, clears the in-memory ontology, then retrieves the namespace document via ?uri= and verifies both class descriptions are returned. The namespace is not DataManager-mapped and not a registered app, so the request exercises the new OntModel DESCRIBE path in ProxyRequestFilter. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Fix ProxyRequestFilterTest after ontology injection added Set filter.ontology = Optional.empty() in setUp() so existing tests reach the proxy/NotAllowed paths they were designed to test. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * Injection fixes --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.